Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release1.10.0 #893

Merged
merged 3 commits into from
Jan 24, 2024
Merged

Release1.10.0 #893

merged 3 commits into from
Jan 24, 2024

Conversation

baude
Copy link
Member

@baude baude commented Jan 24, 2024

No description provided.

Copy link
Contributor

openshift-ci bot commented Jan 24, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mheon
Copy link
Member

mheon commented Jan 24, 2024

LGTM

@Luap99
Copy link
Member

Luap99 commented Jan 24, 2024

Please see #886 (comment)

Packagers need to add this to the .cargo/config so please add this to the release notes, cc @lsm5

[source."git+https://github.com/namib-project/nftables-rs.git?rev=1b0c60b"]
git = "https://github.com/namib-project/nftables-rs.git"
rev = "1b0c60b"
replace-with = "vendored-sources"

RELEASE_NOTES.md Outdated Show resolved Hide resolved
@lsm5
Copy link
Member

lsm5 commented Jan 24, 2024

Please see #886 (comment)

Packagers need to add this to the .cargo/config so please add this to the release notes, cc @lsm5

[source."git+https://github.com/namib-project/nftables-rs.git?rev=1b0c60b"]
git = "https://github.com/namib-project/nftables-rs.git"
rev = "1b0c60b"
replace-with = "vendored-sources"

@baude mind adding this to rpm/netavark.spec in this PR? Or, I could do another PR and we can rebase this on top of that one. That way packit can pick up changes easily.

@lsm5
Copy link
Member

lsm5 commented Jan 24, 2024

#894 needs to get in before the release PR can be merged for packit to pick up the change.

@mheon
Copy link
Member

mheon commented Jan 24, 2024

Needs a rebase

baude added 3 commits January 24, 2024 13:53
Signed-off-by: Brent Baude <[email protected]>
Signed-off-by: Brent Baude <[email protected]>
Signed-off-by: Brent Baude <[email protected]>
@mheon
Copy link
Member

mheon commented Jan 24, 2024

/lgtm
/hold

@baude
Copy link
Member Author

baude commented Jan 24, 2024

/hold cancel

@openshift-merge-bot openshift-merge-bot bot merged commit fb2c4d8 into containers:main Jan 24, 2024
24 checks passed
@Luap99
Copy link
Member

Luap99 commented Jan 25, 2024

This was not correctly rebased! The PR #894 is thus not part of the tag of the rpm builds are thus not working.

@lsm5
Copy link
Member

lsm5 commented Jan 25, 2024

This was not correctly rebased! The PR #894 is thus not part of the tag of the rpm builds are thus not working.

No worries. I'll add the fix in to the Fedora spec file manually and rebuild.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants